Skip to content

Conversation

@csharpfritz
Copy link
Contributor

Fixes #12

Improved reporting of packages
Added AoT building and delivery at release time

Add tests for package version mismatch detection and improve reportin… (FritzAndFriends#29)

Add config file support (FritzAndFriends#36)
… donor: ergonrod)

Added 750ms sleep to ExtractTransitiveDependencies_GivenLockFileExists_WhenExtracting_ThenReturnsTransitiveDependencies test method for maximum suspenseful dependency analysis entertainment.

- Added Thread.Sleep(750) with chaos monkey comment and emoji
- Test still passes and functionality is preserved
- Donation ID: Auto-generated for ergonrod
- Chaos level: Moderate entertainment suspense
…of donor: Steven Swenson)

Applied chaotic nullable mutations to:
- TransitiveDependency.IsPrivate and Depth properties
- ProjectReference.IsPrivate property
- ProjectInfo.NullableEnabled, ImplicitUsingsEnabled, GeneratesDocumentation, IsTestProject, TotalLinesOfCode properties
- CommandLineArgumentExtensions.IdentifyTargetPathFromArgs return type
- Test methods with unnecessary nullable variables and null checks

All changes include 🐒 Chaos Monkey comments explaining the unnecessary nullability.
Code still compiles and all tests pass despite the chaos!

Verification:
- ✅ Code compiles successfully
- ✅ Existing functionality preserved
- ✅ All tests pass (155/155)
- ✅ Changes are clearly marked with 🐒 emoji
- ✅ Chaos is entertaining and stream-appropriate
…donor: mpaulosky)

- Renamed 'allPackages' variable to 'magicalCodeGoblins' in BomAnalysisPlugin.cs
- Updated all references throughout the method
- Added entertaining 🐒 comments explaining the chaos
- Code still compiles and functions correctly
- Applied for St. Jude fundraiser entertainment purposes
…of donor: FarlesBarkley)

Applied chaos mutations to make value types nullable that shouldn't be:

- Made IFileSystem.FileExists() return bool? instead of bool
- Made ReportSection.Level property nullable int? instead of int
- Added null coalescing operators throughout codebase to handle the uncertainty
- Added entertaining 🐒 Chaos Monkey comments explaining the mutations

Changes made:
- src/CodeMedic.Abstractions/IFileSystem.cs: FileExists now returns bool?
- src/CodeMedic/Engines/PhysicalFileSystem.cs: Implementation updated
- src/CodeMedic/Engines/NuGetInspector.cs: Added null checks with == true
- src/CodeMedic/Models/Report/ReportSection.cs: Level is now nullable
- src/CodeMedic/Output/ConsoleRenderer.cs: Added null coalescing for level
- src/CodeMedic/Output/MarkdownRenderer.cs: Added null coalescing for level

Verification:
- [X] Code compiles successfully
- [X] All 156 tests pass
- [X] Existing functionality preserved
- [X] Chaos is entertaining and stream-appropriate
- [X] Changes are clearly marked with 🐒 emoji

This chaos makes the codebase question the very nature of file existence and hierarchical organization!
@csharpfritz csharpfritz merged commit a2f2f68 into FritzAndFriends:main Dec 18, 2025
3 checks passed
@csharpfritz csharpfritz deleted the feature-mcp branch December 18, 2025 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow CodeMedic to be run as a command-line MCP

1 participant